Update semver to 0.2.0

Auto merge of #2134 - alexcrichton:build-script-input, r=brson

Currently Cargo is quite conservative in how it determines whether a build
script should be run. The heuristic used is "did any file in the project
directory change", but this is almost always guaranteed to be too coarse
grained in situations like:

* If the build script takes a long time to run it's advantageous to run it as
  few times as possible. Being able to inform Cargo about precisely when a build
  script should be run should provide more robust support here.
* Build scripts may not always have all of their dependencies in-tree or in the
  crate root. Sometimes a dependency could be elsewhere in a repository and
  scripts need a method of informing Cargo about this (as currently these
  compiles don't happen then they should).

This commit adds this support in build scripts via a new `rerun-if-changed`
directive which can be printed to standard output (using the standard Cargo
metadata format). The value for this key is a path relative to the crate root,
and Cargo will only look at these paths when determining whether to rerun the
build script. Any other file changes will not trigger the build script to be
rerun.

Currently the printed paths may either be a file or a directory, and a directory
is deeply traversed. The heuristic for trigger a rerun is detecting whether any
input file has been modified since the build script was last run (determined by
looking at the modification time of the output file of the build script). This
current implementation means that if you depend on a directory and then delete a
file within it the build script won't be rerun, but this is already the case and
can perhaps be patched up later.

Future extensions could possibly include the usage of glob patterns in build
script paths like the `include` and `exclude` features of `Cargo.toml`, but
these should be backwards compatible to add in the future.

Closes #1162

Auto merge of #2190 - sfackler:wildcard-pushback, r=alexcrichton

We missed a release somewhere, so the warning here won't hit stable
until December 11th.

r? @alexcrichton

Move wildcard rejection date back 6 weeks

We missed a release somewhere, so the warning here won't hit stable
until December 11th.

Auto merge of #2186 - alexcrichton:transaction, r=brson

Right now a `Bomb` struct is used to attempt to ensure that broken tarballs
don't escape, but this unfortunately doesn't work for when Cargo is terminated
via other means such as ctrl-c or abnormal termination. Instead the tarball is
constructed in a temporary location and then only moved to the final location
once all checks pass.

Closes #2173
cc #2177

Auto merge of #2185 - alexcrichton:update, r=alexcrichton

Update plugin tests to nightly

Auto merge of #2184 - brandonson:master, r=alexcrichton

When the crate to install is unspecified, we should really inform users about
where they can install from (other than crates.io).  This makes the message mention --path
and --git so that people know they exist.

Package/test tarballs in a temporary location

Right now a `Bomb` struct is used to attempt to ensure that broken tarballs
don't escape, but this unfortunately doesn't work for when Cargo is terminated
via other means such as ctrl-c or abnormal termination. Instead the tarball is
constructed in a temporary location and then only moved to the final location
once all checks pass.

Closes #2173
cc #2177

Improve message with unspecified install target.

When the crate to install is unspecified, we should really inform users about
where they can install from (other than crates.io).  This makes the message mention --path
and --git so that people know they exist.

Auto merge of #2176 - gentoo90:bash-comp, r=alexcrichton

* completion for `install`, `uninstall` and `rustdoc`
* completion for `--color` variants
* some missing options

Update bash-completion

* completion for install, uninstall and rustdoc
* completion for --color variants
* some missing options

Auto merge of #2168 - durka:patch-3, r=alexcrichton

In c9f1b9bf1e535a651f624897bd53a3f81af7501f the format of this example was changed from a `[dependencies.awesome]` section to the newly-recommended inline table syntax, but one of the attributes was left out.

I had to use some slightly weird formatting to keep the comment in place, is that OK?

restore dependencies.awesome normal table

inline tables must be on one line, because TOML

doc/manifest: fix "default-features" example

Auto merge of #2167 - iirelu:vec-macro-consistency, r=alexcrichton

~~Because it bugged me~~ It's better to have a single consistent way of using macros, and not mixing `{}` `[]` `()` blocks.

Make usage of vec! macro consistent

Previously it was a mix of vec!() and vec![], as both work thanks to the
implementation of macros in Rust (even vec!{} would work), this makes
them all use the generally-accepted vec![] standard, which imitates the
syntax used for slices and is considered to make things clearer.

Auto merge of #2166 - alexcrichton:clean-up-update, r=huonw

I've noticed some slightly odd output from `cargo update` in the past and I
believe this cleanup should address what's going on under the hood. There were a
few minor issues with the previous implementation.

* When adding the previous graph to the list of changes, packages with multiple
  versions would override one another instead of all get added to one list.
* The `Ord` implementation for `SourceId` was actually incorrect in that it
  disagreed with the `Eq` implementation. This could end up causing subtle bugs
  here and there. dependening on what operators were used.

This tweak fixes both points and I believe should touch up the odd output I've
been seeing from `cargo update`.

Clean up the `cargo update` implementation a bit

I've noticed some slightly odd output from `cargo update` in the past and I
believe this cleanup should address what's going on under the hood. There were a
few minor issues with the previous implementation.

* When adding the previous graph to the list of changes, packages with multiple
  versions would override one another instead of all get added to one list.
* The `Ord` implementation for `SourceId` was actually incorrect in that it
  disagreed with the `Eq` implementation. This could end up causing subtle bugs
  here and there. dependening on what operators were used.

This tweak fixes both points and I believe should touch up the odd output I've
been seeing from `cargo update`.

Auto merge of #2160 - alexcrichton:bail-macro, r=huonw

This pattern showed up quite a few locations throughout the codebase but it
ended up meaning that there were some massive levels of indentation when you
actually get to the formatting string. This commit adds a new `bail!` macro
which shortens this idiom to help get the indentation under control throughout
the codebase.

Add a macro for `return Err(human(format!(...)))`

This pattern showed up quite a few locations throughout the codebase but it
ended up meaning that there were some massive levels of indentation when you
actually get to the formatting string. This commit adds a new `bail!` macro
which shortens this idiom to help get the indentation under control throughout
the codebase.

Auto merge of #2159 - Manishearth:native-diag, r=alexcrichton

r? @alexcrichton

Improve diagnostics for multiple native links for the same package

Auto merge of #2156 - alexcrichton:rebase-2058, r=alexcrichton

Touching up a few last comments before landing

Touch up a few last minor comments

Merge branch 'cwd-fixes' of https://github.com/Kimundi/cargo

Auto merge of #2154 - alexcrichton:update, r=alexcrichton

Closes #2152

Auto merge of #2153 - carlosgaldino:cg-fix-uninstall, r=alexcrichton

With this patch users can type `cargo help uninstall` or `cargo
uninstall -h` and it will display the `USAGE` message as it was supposed to.

Before the patch the command would display an error message about
invalid arguments.

Fixes #2138.

I also updated the man page.

Update man page.

Fix `cargo-uninstall`.

With this patch users can type `cargo help uninstall` or `cargo
uninstall -h` and it will display the `USAGE` message as it was supposed
to.

Before the patch the command would display an error message about
invalid arguments.

Fixes #2138.

Update dependencies

Closes #2152

Auto merge of #2151 - IvanUkhov:typography, r=steveklabnik

Hi,

Here are a number of commits making cosmetic adjustments to the documentation. The major change concerns the usage of quotation marks. If needed, some of the commits can be removed, and the rest can be squashed into one. Thanks.

Regards,
Ivan

Spell “Cargo” consistently

Spell “GitHub” consistently

Fix a typo

Use “etc.” consistently

Use proper quotation marks

Auto merge of #2131 - alexcrichton:cargo-clean, r=brson

* Clean out both host/target platforms if any are available.
* Clean out build script output.
* Add a --release flag to also be able to clean out release folder
* Be sure to clean out benchmarks, tests, etc.
* Cleaning out registry packages no longer panics

Closes #2121

Fix a number of bugs with cargo clean

* Clean out both host/target platforms if any are available.
* Clean out build script output.
* Add a --release flag to also be able to clean out release folder
* Be sure to clean out benchmarks, tests, etc.
* Cleaning out registry packages no longer panics

Closes #2121

Auto merge of #2136 - gkoz:empty_fingerprint, r=alexcrichton

None

Fix #1504. Don't blow up on empty fingerprint files

Allow build scripts to specify dependencies

Currently Cargo is quite conservative in how it determines whether a build
script should be run. The heuristic used is "did any file in the project
directory change", but this is almost always guaranteed to be too coarse
grained in situations like:

* If the build script takes a long time to run it's advantageous to run it as
  few times as possible. Being able to inform Cargo about precisely when a build
  script should be run should provide more robust support here.
* Build scripts may not always have all of their dependencies in-tree or in the
  crate root. Sometimes a dependency could be elsewhere in a repository and
  scripts need a method of informing Cargo about this (as currently these
  compiles don't happen then they should).

This commit adds this support in build scripts via a new `rerun-if-changed`
directive which can be printed to standard output (using the standard Cargo
metadata format). The value for this key is a path relative to the crate root,
and Cargo will only look at these paths when determining whether to rerun the
build script. Any other file changes will not trigger the build script to be
rerun.

Future extensions could possibly include the usage of glob patterns in build
script paths like the `include` and `exclude` features of `Cargo.toml`, but
these should be backwards compatible to add in the future.

Closes #1162

Auto merge of #2135 - alexcrichton:bundle-crate-in-registry, r=huonw

Even if multiple ones are included, don't recurse!

Closes #2132

Only read one package from registry tarballs

Even if multiple ones are included, don't recurse!

Closes #2132

Refactor testing the registry

Use a builder to publish packages instead of a one-off function with many
variants, should provide a much finer grained level of configurability for
future test cases against the registry.

Auto merge of #2128 - k0pernicus:patch-1, r=alexcrichton

Add a comment to know how to pass arguments via Cargo

Update guide.md

Modification of the second sentence, to explain how to pass flags with Cargo

Auto merge of #2126 - johshoff:default_dependency_header, r=alexcrichton

Almost any project beyond "hello world" will have some dependencies.
Including the `[dependencies]` header by default makes this slightly
simpler.

For new users of the language, this can potentially save some
frustration since crates.io currently does not mention the header, just
the line that goes beneath it.

For all users, this makes adding the first dependency to a project less
of a special case than to subsequent dependencies.

Update guide.md

Correctness of the previous modification, regarding how to pass arguments via Cargo

Auto merge of #2129 - alexcrichton:cargo-rustdoc, r=brson

Along the same lines as `cargo rustc` basically

Update guide.md

Add a comment to know how to pass arguments via Cargo

Rebase `cargo rustdoc` and tweak behavior

This rebases #1977 onto master and also tweaks the behavior to match `cargo
rustc` when there are multiple targets in play.

Add `cargo rustdoc` for passing arbitrary flags to rustdoc

Add dependency header in default Cargo.toml

Almost any project beyond "hello world" will have some dependencies.
Including the `[dependencies]` header by default makes this slightly
simpler.

For new users of the language, this can potentially save some
frustration since crates.io currently does not mention the header, just
the line that goes beneath it.

For all users, this makes adding the first dependency to a project less
of a special case than to subsequent dependencies.

Auto merge of #2118 - alexcrichton:update-curl, r=alexcrichton

This should help continue to mitigate #1420 further

Update to a curl-rust that doesn't use OpenSSL on OSX

This should help continue to mitigate #1420 further

Auto merge of #2115 - alexcrichton:nondeterminism, r=brson

These were all exhibiting some nondeterminism which meant that they were failing
tests on CI.

Relax some output assertions in tests

These were all exhibiting some nondeterminism which meant that they were failing
tests on CI.

Auto merge of #2114 - alexcrichton:install-relative-path, r=brson

Closes #2106

Fix install `--path` with a relative path

Closes #2106

Auto merge of #2113 - alexcrichton:weird-filenames, r=brson

This commit is targeted at fixing #2102 via two routes:

1. The dependency on `tar` was upgraded to include more contextual information
   in error messages about why the unpack failed. This should help diagnose
   these sorts of issues that happen in the first place.
2. Packaging crates that have files with odd filenames is no longer allowed.
   An error is returned indicating that the files cannot be packaged as they're
   not cross platform. The currently rejected set of files are non-utf8
   filenames (already present) and those containing characters special on
   Windows.

Closes #2102

Add more safeguards around odd filenames

This commit is targeted at fixing #2102 via two routes:

1. The dependency on `tar` was upgraded to include more contextual information
   in error messages about why the unpack failed. This should help diagnose
   these sorts of issues that happen in the first place.
2. Packaging crates that have files with odd filenames is no longer allowed.
   An error is returned indicating that the files cannot be packaged as they're
   not cross platform. The currently rejected set of files are non-utf8
   filenames (already present) and those containing characters special on
   Windows.

Closes #2102

Auto merge of #2091 - winger:master, r=alexcrichton

API changes needed for https://github.com/winger/cargo-metadata external command.
See https://github.com/rust-lang/cargo/pull/1434 for previous discussion.

Changes required for cargo-metadata

Auto merge of #2103 - takkanm:add_install_help, r=alexcrichton

There was no "install" to help message.

add install help

Auto merge of #2104 - mbrubeck:test-doc, r=alexcrichton

None

Docs: Mention the [[example]] section

Expand documentation on integration tests

Docs: Clarify what `cargo test` compiles vs. runs

Auto merge of #2097 - alexcrichton:dont-panic-when-slicing, r=huonw

Closes #2094

Don't panic when printing the precise source id

Closes #2094

Auto merge of #2092 - gkoz:lock_line_endings, r=alexcrichton

Before writing the lockfile read the existing one and

 * detect the line endings style, fixes #2076, #1722;
 * compare the contents to avoid redundant overwriting.

Detect the line endings convention of Cargo.lock (CRLF or LF) and preserve it

Fixes #2076

Avoid overwriting the lockfile if there are no changes

Auto merge of #2093 - alexcrichton:bump, r=alexcrichton

Bump to 0.7.0

Add a test for #2076, preservation of line endings style

Auto merge of #2070 - alexcrichton:test-optional, r=brson

Previously a warning was issued if both -p and --features were passed as flags,
and the rationale for this was that if --features modified the activated set of
features in the package selected by -p it would alter Cargo.lock, which is
undesirable as Cargo.lock should be stable.

This commit, however, interprets --features as changing the resolved graph of
the top-level package, and then -p is a query on that resolved graph. This way
the Cargo.lock file never changes and you're allowed to test optional
dependencies.

Closes #2083

Fixed testsuite

Made cwd setting in process builder optional

Made cargo derive more working directory locations from its Config

Auto merge of #2077 - Lechindianer:master, r=alexcrichton

Just a simple fix

Fixed typo

Auto merge of #2061 - alexcrichton:update, r=alexcrichton

* Move along the same rails for all dependencies, picking up various small perf,
  build, and portability improvements.
* Update pinned rustc to pick up perf improvements and such
* Tweak expected error message from tests to continue to work

Update dependencies

* Move along the same rails for all dependencies, picking up various small perf,
  build, and portability improvements.
* Update pinned rustc to pick up perf improvements and such
* Tweak expected error message from tests to continue to work

Auto merge of #2071 - aarzee:master, r=alexcrichton

Whitespace: remove leading newlines; replace lines containing only whitespace with empty lines; replace multiple trailing newlines with a single newline; remove trailing whitespace in lines.

Images: Compress PNG files with zopflipng, JPG files with mozjpeg jpegtran, and GIF files with gifsicle.

This PR was created semiautomatically.

Auto merge of #2072 - durka:patch-2, r=alexcrichton

fix copy/paste error in doc

Cleanup

Whitespace: remove leading newlines; replace lines containing only whitespace with empty lines; replace multiple trailing newlines with a single newline; remove trailing whitespace in lines. Images: Compress PNG files with zopflipng, JPG files with mozjpeg jpegtran, and GIF files with gifsicle.

Allow testing optional dependencies

Previously a warning was issued if both -p and --features were passed as flags,
and the rationale for this was that if --features modified the activated set of
features in the package selected by -p it would alter Cargo.lock, which is
undesirable as Cargo.lock should be stable.

This commit, however, interprets --features as changing the resolved graph of
the top-level package, and then -p is a query on that resolved graph. This way
the Cargo.lock file never changes and you're allowed to test optional
dependencies.

Auto merge of #2068 - alexcrichton:root-profiles, r=brson

When testing or building multiple packages via the `-p` argument the root
package's profiles should always be used instead of the sub-package's set of
profiles.

Always use the root package's set of profiles

When testing or building multiple packages via the `-p` argument the root
package's profiles should always be used instead of the sub-package's set of
profiles.

Auto merge of #2067 - apasel422:spell, r=alexcrichton

Correct spelling in docs

Auto merge of #2062 - aarzee:master, r=steveklabnik

Remove leading newlines; replace lines containing only whitespace with empty lines; replace multiple trailing newlines with a single newline; remove trailing whitespace in lines.

This PR was created semiautomatically.

Clean whitespace

Remove leading newlines; replace lines containing only whitespace with empty lines; replace multiple trailing newlines with a single newline; remove trailing whitespace in lines

Auto merge of #2026 - alexcrichton:cargo-install, r=brson

This commit is an implementation of [RFC 1200][rfc] which brings two new
subcommands: `cargo install` and `cargo uninstall`. Most of this is a straight
implementation of the RFC, but a few tweaks were made:

* The `-p` or `--package` arguments are no longer needed as you just pass
  `crate` as a bare argument to the command, this means `cargo install foo`
  works and downloads from crates.io by default.
* Some logic around selecting which crate in a multi-crate repo is installed has
  been tweaked slightly, but mostly in the realm of "let's do the thing that
  makes sense" rather than the literal "let's do what's in the RFC".
  Specifically, we don't pick a crate with examples if there are multiple crates
  with binaries (instead an error is generated saying there are multiple binary
  crates).

[rfc]: https://github.com/rust-lang/rfcs/blob/master/text/1200-cargo-install.md

Finish implementing `cargo install`

This commit is an implementation of [RFC 1200][rfc] which brings two new
subcommands: `cargo install` and `cargo uninstall`. Most of this is a straight
implementation of the RFC, but a few tweaks were made:

* The `-p` or `--package` arguments are no longer needed as you just pass
  `crate` as a bare argument to the command, this means `cargo install foo`
  works and downloads from crates.io by default.
* Some logic around selecting which crate in a multi-crate repo is installed has
  been tweaked slightly, but mostly in the realm of "let's do the thing that
  makes sense" rather than the literal "let's do what's in the RFC".
  Specifically, we don't pick a crate with examples if there are multiple crates
  with binaries (instead an error is generated saying there are multiple binary
  crates).

[rfc]: https://github.com/rust-lang/rfcs/blob/master/text/1200-cargo-install.md